Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Workaround: Separate saved searches with tags name #703

Merged

Conversation

Neui
Copy link
Contributor

@Neui Neui commented Sep 5, 2021

Since unfortunately saved searches and tags share the same namespace, it
makes it difficult to create a saved search with the exact name as an
existing tag.

As a current workaround until we separate them, we just prefix the saved
searches with SEARCH_TAG_PREFIX, which an user is unlikely to use.
From what I can see, it still displays properly in the UI after some
fixes, aside from existing bugs, like the tag editor stripping spaces, @blog !or @efficiency having the first @ stripped out in the sidebar, and so on.

In the XML file this prefix isn't added, because it would be redundant
and make it easier to deal with in future versions.

See for example #629. Does not fix tag inconsistencies (renaming, special symbols etc.).

Since unfortunately saved searches and tags share the same namespace, it
makes it difficult to create a saved search with the exact name as an
existing tag.

As a current workaround until we separate them, we just prefix the saved
searches with SEARCH_TAG_PREFIX, which an user is unlikely to use.
From what I can see, it still displays properly in the UI after some
fixes, aside from existing bugs.

In the XML file this prefix isn't added, because it would be redudant
and make it easier to deal with in future versions.
@diegogangl diegogangl added this to the 0.6 (The future) milestone Nov 15, 2021
@diegogangl diegogangl merged commit 14239a9 into getting-things-gnome:master Nov 17, 2021
@diegogangl
Copy link
Contributor

Always impressed by how thorough your are with these PRs, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants